Skip to content

Touched up the comments a bit#31

Open
SwiftsNamesake wants to merge 1 commit intoTwinside:masterfrom
SwiftsNamesake:master
Open

Touched up the comments a bit#31
SwiftsNamesake wants to merge 1 commit intoTwinside:masterfrom
SwiftsNamesake:master

Conversation

@SwiftsNamesake
Copy link

Sorry about the auto-formatting stuff, I'll fix it if it's too much.

@Twinside
Copy link
Owner

Hi, sorry I didn't had time to reply to the clarification ticket, some remarks:

  • Stroking is an accepted term in the domain, I wouldn't change it
  • Will -> Fill doesn't makes sense, maybe just removing the word would work.
  • I'm don't like the alignment of imports (a pita to maintain it) and various free indentation changes, as it breaks the history.

Otherwise, thanks, I see that I made a lot of typos :]

@SwiftsNamesake
Copy link
Author

SwiftsNamesake commented Sep 20, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants